Skip to content

fix(graph): optimize the size of the serialized schema#182

Merged
averikitsch merged 4 commits into
googleapis:mainfrom
jordimas:issue_181
Jul 22, 2025
Merged

fix(graph): optimize the size of the serialized schema#182
averikitsch merged 4 commits into
googleapis:mainfrom
jordimas:issue_181

Conversation

@jordimas
Copy link
Copy Markdown
Contributor

Related to #181

@jordimas jordimas requested review from a team May 31, 2025 14:57
@product-auto-label product-auto-label Bot added the api: spanner Issues related to the googleapis/langchain-google-spanner-python API. label May 31, 2025
@jordi-mas-dj
Copy link
Copy Markdown

cc: @averikitsch @gauravpurohit06

@averikitsch
Copy link
Copy Markdown
Collaborator

/gcbrun

@averikitsch
Copy link
Copy Markdown
Collaborator

averikitsch commented Jul 21, 2025

What is the affect of this PR i.e. is it only for printing out the schema or does it break previous functionality related to creating graph schemas? Thanks!

@jordimas
Copy link
Copy Markdown
Contributor Author

Hi! It only changes how the schema is serialized that is only used to be included in prompt via {schema} variable.
It does not break previous functionality.
There is more background here: #181
Thanks

@averikitsch
Copy link
Copy Markdown
Collaborator

/gcbrun

@averikitsch averikitsch merged commit 51dba31 into googleapis:main Jul 22, 2025
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: spanner Issues related to the googleapis/langchain-google-spanner-python API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants